Skip to content

Conversation

@lilac
Copy link
Contributor

@lilac lilac commented Feb 4, 2014

Closes #11630.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe these could all be covered by the same gate (e.g. called quote)?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I didn't think about it. It seems that there was only one "quote" macro, but later split to multiple ones.
I am not sure if there is a gain to keep them separate in feature gate as well.
If not, I am happy to replace with a single gate, that is simpler.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes to the single, simple quote feature gate.

@brson
Copy link
Contributor

brson commented Feb 4, 2014

Thanks!

bors added a commit that referenced this pull request Feb 5, 2014
@bors bors closed this Feb 5, 2014
@bors bors merged commit 124938b into rust-lang:master Feb 5, 2014
@lilac lilac deleted the feature-gate-quote branch February 5, 2014 10:49
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 25, 2022
minor: Document completion context some more
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jan 11, 2024
Include GitLab in the CI section of the clippy doc book

Fixes rust-lang#12012
changelog: Docs: [`Continuous Integration`] now includes how to use clippy in GitLab CI.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feature gate all quasi-quoting macros

4 participants